Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AllDialogからHelpDialogを切り出す #2419

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jdkfx
Copy link
Contributor

@jdkfx jdkfx commented Dec 17, 2024

内容

  • HelpDialogについて、useDialogPluginComponentを使って、AllDialogから切り出してみる

関連 Issue

ref #1551

スクリーンショット・動画など

その他

  • 実験的にやってみた内容なので、クローズするかもしれません。
  • エラーが出ているので修正しなければいけないのだが、よく分からず止まっている。
  • メニューバーからヘルプを開いた際に、一番最初のソフトウェアの利用規約部分が真っ白になっていた。
スクリーンショット 2024-12-18 0 06 33

@voicevox-preview-pages
Copy link

voicevox-preview-pages bot commented Dec 17, 2024

🚀 プレビュー用ページを作成しました 🚀

更新時点でのコミットハッシュ:a0ea297

@madosuki
Copy link
Contributor

https://github.com/jdkfx/voicevox/blob/refactoring/%231551_use_dialog_plugin_component/src/components/Dialog/HelpDialog/HelpDialog.vue#L131C1-L132C74

const policy = ref<string>();
void store.actions.GET_POLICY_TEXT().then((obj) => (policy.value = obj));

ここのところref()だと初期値はundefinedになっていて更新は非同期関数で行われています。恐らく読み込みが間に合っていない状態でHelpMarkdownViewSection.vueに渡されているためエラーが出たのだと思います。
なので、ref()になっているところをref("読み込み中")など適当な文字列で初期化するようにした上で、HelpMarkdownViewSection.vue内でprops.markdownをwatchして更新処理を入れれば行けるかもしれません。

@jdkfx
Copy link
Contributor Author

jdkfx commented Dec 18, 2024

ここのところref()だと初期値はundefinedになっていて更新は非同期関数で行われています。恐らく読み込みが間に合っていない状態でHelpMarkdownViewSection.vueに渡されているためエラーが出たのだと思います。

ありがとうございます!解決しました!

@jdkfx jdkfx marked this pull request as ready for review December 18, 2024 04:32
@jdkfx jdkfx requested a review from a team as a code owner December 18, 2024 04:32
@jdkfx jdkfx requested review from Hiroshiba and removed request for a team December 18, 2024 04:32
@jdkfx
Copy link
Contributor Author

jdkfx commented Dec 18, 2024

@madosuki

ついでにこれらも同様に初期化するようにしておいた方が良いかと思います。

console上では特に問題なさそうだったので修正を入れてませんでしたが、一応入れておきます!ありがとうございます!

@jdkfx jdkfx changed the title useDialogPluginComponentでAllDialogから一部移動してみる AllDialogからHelpDialogを切り出す Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants